Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[build][Zig] Update replaced defineCMacro() with equivalent root_module.addCMacro(). #4629

Closed
wants to merge 2 commits into from

Conversation

pierrelgol
Copy link

This pr updates the build.zig files in the root directory and in the example directory. I don't think this needs to be merged right now, but, this is the change that I needed to do locally to get raylib to compile with the nightly release of the Zig compiler.

@pierrelgol pierrelgol changed the title Build zig update replaced 'defineCMacro() with equivalent 'root_module.addCMacro(). Build zig update replaced defineCMacro() with equivalent root_module.addCMacro(). Dec 21, 2024
@HaxSam
Copy link
Contributor

HaxSam commented Dec 21, 2024

There is already a pr for that
look #4620

@raysan5 raysan5 changed the title Build zig update replaced defineCMacro() with equivalent root_module.addCMacro(). [build] Zig update replaced defineCMacro() with equivalent root_module.addCMacro(). Dec 23, 2024
@raysan5 raysan5 changed the title [build] Zig update replaced defineCMacro() with equivalent root_module.addCMacro(). [build][Zig] Update replaced defineCMacro() with equivalent root_module.addCMacro(). Dec 23, 2024
@raysan5
Copy link
Owner

raysan5 commented Dec 26, 2024

It seems this PR is a duplicate of referred one, closing this one.

@raysan5 raysan5 closed this Dec 26, 2024
@pierrelgol pierrelgol deleted the build_zig_update branch December 29, 2024 08:26
@pierrelgol
Copy link
Author

My bad I didn't even saw that someone already made a pr, sorry for disturbing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants